Skip to content

[Gardening] Resolve deprecated API use warnings and remove stale code #1892

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented May 8, 2025

  • Deprecate old SwiftScan global scanner diagnostic queries. We have since moved on to per-scan queries and the approach being deprecated here is not safe.
  • Replace occurrences of the deprecated ProcessEnv.vars with ProcessEnv.block
  • Deprecate enablement of explicit modules with an environment variable
  • Resolve warnings about deprecated AbsolutePath initializer
  • Migrate ProcessSet implementation to the driver
  • Cleanup the README.md to remove stale content

@artemcm
Copy link
Contributor Author

artemcm commented May 8, 2025

@swift-ci test

1 similar comment
@artemcm
Copy link
Contributor Author

artemcm commented May 8, 2025

@swift-ci test

@artemcm
Copy link
Contributor Author

artemcm commented May 14, 2025

@swift-ci test

@artemcm artemcm force-pushed the GardenEBM branch 3 times, most recently from c58a5b6 to c61b973 Compare May 14, 2025 19:42
@artemcm
Copy link
Contributor Author

artemcm commented May 14, 2025

@swift-ci test

artemcm added 6 commits May 14, 2025 12:50
This type is being deprecated and removed in 'swift-tools-support-core'. Move the implementation to the driver.
Long term, the driver's executors should be re-architected to rely on structured concurrency primitives. In the meantime, absorb this functionality into the driver so that it can be freely removed from STSC.
@artemcm
Copy link
Contributor Author

artemcm commented May 14, 2025

@swift-ci test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant